Skip to content
This repository has been archived by the owner on Sep 21, 2024. It is now read-only.

fix: Get chromedriver from a new location #514

Merged
merged 1 commit into from
Jul 19, 2023

Conversation

cdata
Copy link
Collaborator

@cdata cdata commented Jul 19, 2023

Our workflow script for getting chromedriver for our browser-targeted tests broke from under us due to changes in the Chromium project's release processes (see: https://groups.google.com/g/chromedriver-users/c/clpipqvOGjE).

We originally cribbed the script from /~https://github.com/SeleniumHQ/selenium/blob/5d108f9a679634af0bbc387e7e3811bc1565912b/.github/actions/setup-chrome/action.yml

However, Selenium has moved to using /~https://github.com/browser-actions/setup-chrome which does not handle setting up chromedriver.

The original mechanism we depended on to discover the correct chromedriver version has been replaced by the static JSON published in /~https://github.com/GoogleChromeLabs/chrome-for-testing#json-api-endpoints

So, this change adapts our workflow scripts to use the new mechanism to setup chromedriver.

@cdata cdata requested review from jsantell and justinabrahms July 19, 2023 19:46
@cdata cdata force-pushed the fix/chrome-setup-in-gh-workflows branch 3 times, most recently from 9fbc707 to c10f465 Compare July 19, 2023 19:55
Copy link
Contributor

@jsantell jsantell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants